Skip to content

Keep gx doctor visibly progressing through nested repo repairs#230

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/improve-gx-doctor-output-and-speed-2026-04-21-14-15
Apr 21, 2026
Merged

Keep gx doctor visibly progressing through nested repo repairs#230
NagyVikt merged 1 commit intomainfrom
agent/codex/improve-gx-doctor-output-and-speed-2026-04-21-14-15

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by scripts/agent-branch-finish.sh (PR flow).

Recursive doctor now streams nested child output live, annotates each target with progress and completion timing, respects doctor --no-wait-for-merge inside the auto-finish sweep, and compacts the default sweep failures so long rebase command traces stop overwhelming the terminal. Verbose mode preserves the raw detail when needed.

Constraint: Doctor still uses synchronous child execution, so responsiveness had to come from live inherited output and shorter summaries instead of a larger async rewrite

Rejected: Leave recursive doctor buffered and only change the copy | it would still look frozen in large workspaces

Rejected: Hide all auto-finish failure details by default | users still need branch-level actionable reasons without rerunning the command blindly

Confidence: high

Scope-risk: moderate

Reversibility: clean

Directive: Keep doctor JSON output stable; only the human-readable doctor path should compact or reorder auto-finish details

Tested: node --check bin/multiagent-safety.js; node --test --test-name-pattern "doctor" test/install.test.js; openspec validate agent-codex-improve-gx-doctor-output-and-speed-2026-04-21-14-15 --type change --strict; openspec validate --specs; git diff --check

Not-tested: Full install test suite outside the doctor-focused slice
@NagyVikt NagyVikt merged commit bac4805 into main Apr 21, 2026
@NagyVikt NagyVikt deleted the agent/codex/improve-gx-doctor-output-and-speed-2026-04-21-14-15 branch April 21, 2026 12:34
NagyVikt added a commit that referenced this pull request Apr 21, 2026
#231)

The doctor output/performance branch already merged as PR #230 and its sandbox is gone, so this follow-up updates the existing OpenSpec tasks file with the final merge and cleanup evidence instead of leaving the completion section open.

Constraint: The guarded finish flow merged and cleaned the branch in one pass, but the change bundle still needed an explicit post-merge completion record on main

Rejected: Leave the completion checklist open and rely only on terminal handoff text | this repo expects the OpenSpec tasks file to carry the final merge and cleanup evidence

Confidence: high

Scope-risk: narrow

Reversibility: clean

Directive: When a change bundle merges before its 4.x completion section is updated, add a narrow evidence-only follow-up to that existing tasks file

Tested: openspec validate agent-codex-improve-gx-doctor-output-and-speed-2026-04-21-14-15 --type change --strict; git diff --check

Not-tested: Runtime/test suites, because this is a documentation-only completion-evidence update

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant